add subgroup analysis to hillstrom#101
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a new “Subgroup Analysis by Purchase History” section to the Hillstrom tutorial to illustrate heterogeneous campaign effects by prior purchase category, along with supporting visualization assets.
Changes:
- Introduces subgroup definitions based on
mens/womenspurchase history and computes subgroup ATE/DTE/PTE comparisons. - Adds narrative interpretation and new plots for subgroup DTE/PTE results.
- Adds a static image asset used by the new tutorial section.
Reviewed changes
Copilot reviewed 1 out of 6 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| docs/source/tutorials/hillstrom.rst | Adds subgroup ATE/DTE/PTE walkthrough and embeds subgroup plot images. |
| docs/source/_static/hillstorm_subgroup_women_pte.png | Adds the subgroup PTE plot image referenced by the tutorial. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -0,0 +1,783 @@ | |||
| { | |||
There was a problem hiding this comment.
There's a typo in the filename, can we rename this to hillstorm.ipynb?
There was a problem hiding this comment.
Thanks! The current name hillstrom.ipynb is actually correct.
https://blog.minethatdata.com/2008/03/minethatdata-e-mail-analytics-and-data.html
TomeHirata
left a comment
TomeHirata
left a comment
There was a problem hiding this comment.
LGTM once comments are addressed!
|
Thanks for the comments! I’ve taken care of most of them. |
Add subgroup analysis to hillstrom tutorial.